-
Notifications
You must be signed in to change notification settings - Fork 98
Feat: file validation #90
base: master
Are you sure you want to change the base?
Feat: file validation #90
Conversation
Hi Musab, thanks a lot for contributing! Looks really useful, I'll just need to do a deeper check on this before merging. By the way, does this validation check for specific file extensions or just MIME type? |
Hi Nick,
It checks for specific file types.
|
Hey, thanks again for making this change. There are a couple tweaks I would need before we can merge this:
I can make the necessary commits to augment your PR (no timing guarantee of course) or if you want to do it yourself then feel free as well. Thanks! |
I think only file format should be accepted is csv files. Because(#79 (comment)) That can be achieved that PR as I see or using
|
@unframework could you please deploy it? |
Any chance we can get this deployed? |
@MusabShakeel576 it looks like there are conflicts in this PR. Could you resolve those please so we could maybe get this merged? |
Hey everyone, looks like this feature is getting continued attention, thanks for all the suggestions. There are still some small issues outstanding with the PR (see my earlier comment) and as @dwene pointed out it has a conflict now. I will try to find some time this week to tweak the PR. |
I fixed the PR conflicts in a fork if you want to use any of that code. |
Summary:
Allows specific file formats to be previewed.
Issue:
#79
Checklist